-
Notifications
You must be signed in to change notification settings - Fork 7
Implementation: reference ICAO conversion, with addition of cyrillic and arabic #290
Implementation: reference ICAO conversion, with addition of cyrillic and arabic #290
Conversation
The code implements a change detection on the name input fields and prefills the standardized name when reading the QR-Code see corona-warn-app#185
@janhoffmann @ggrund-tsi |
Hi @thinkberg , As Ubirch already has an implementation for ICAO transliteration for the vaccination DCCs, it would be the biggest gain, if the here proposed PR is aligned as best as possible to your implementation. Therefore, I kindly ask you to have a review on the here proposed solution in terms of completeness and conformity to the ICAO algorithm. In any case, please leave a short line, whether a review from your side is possible. Thank you very much in advance! /cc: @mlenkeit @thomasaugsten @oliver-steinbrecher |
Hi all, |
Hi all again, To enable @janhoffmann to correct some functional problems with the proposed PR, he asked whether it's possible to get him a working enviroment for testing. |
@vaubaehn this here is really frustrating. We have a working (afaik) solution which would improve the work of all stations immediately. Even if it's not matching Ulbrichts implentation this is not a good way to handle this. If we have a difference in some implementation details it will probably only affect a very small percentage of cases. I already saw stations hacking chrome plugins to bypass this long lasting problem. I offered my help on July 2021 and got hardly any feedback for months. After the hint to the DCC implementation guide we did this work in 1 day with a second revision within 1 extra day. Is there anyone at T-Systems, Ulbricht, SAP caring for this?! |
@janhoffmann I fully agree. Insufficient communication from the stakeholders towards the community let us feel ignored, and valuable volunteer contributions sometimes seem to rot in issues and open PRs. I observe that some parts of the most active community is losing their motivation to further continue participating in this project (me included).
It's up to TSI solely. SAP can't do much, as the issue handled here is not in their scope - except raising this issue in their meetings with TSI. Ubirch could open source their ICAO implementation, but they're quite busy with other things like the DCC re-issuing API currently. And their ICAO implementation would help, but getting the RAT portal enhanced with your PR has definetly higher priority, imho - particularly as incidence rates are rising again, and more tests are conducted.
In many cases stakeholders see the responsibility at RKI/BMG, for example with the overcrowded and in some cases low quality TAN hotline (corona-warn-app/cwa-hotline#14), even call centers are contracted by TSI. In my personal opinion, some quality assurance should be undertaken from TSI, as all these problems will fall back to TSI and damage their reputation (despite RKI would need to fund capacities). We can only guess, why the PR here is ignored. Whether RKI is not funding enough to pay one TSI dev to have a look into, whether RKI is not funding enough to have enough staff at TSI to be more flexible to maintain their services, or whether there is just no interest and the community is seen as stupid nerds. The only thing I can think about to speed things up is to make problems visible to the public, e.g. contacting public media. Unfortunately, reports published in media like ComputerBild or Chip/Focus Online have more power to damage the project's reputation than helping to get things done. I'm sorry, I think I can't and won't do more anymore. |
Just one last word to TSI: |
Hi @timokoenig , |
I wasn't event aware that this was a private repository. I removed the links because even I just got removed from that group. At least someone is watching this PR silently 😄 Sad to hear but I can understand your point. Thank you too for all the support! |
Hopefully this is not a Goodbye but only a see you soon! 😢 @vaubaehn Thanks for your continuous support, we (the community) really appreciate it! |
We have this planned, but as @vaubaehn explained we are a bit under right now. |
I will try to at least get the docs out as we did take the comments into account and improved the conversion. I will check over the weekend. |
Thank you! |
For this repo, I'm finished. I will do some housekeeping in other repos, await CWA 2.19.1, and after that there will be a proper good-bye over at CWA 😉 |
FYI: https://github.com/corona-warn-app/cwa-icao-transliteration No idea if this helps here. |
@Ein-Tim great so see progress, but this only shows why nobody should make a effort to contribute here. The implementation of @ggrund-tsi is the tsi way of saying "screw the open source community, we code it ourself". I'm not sad because my work will never be used, I'm sad because it was ignored during the development of their own solution. To sum up: next time I will code my own interface and register as a API partner, this is a joke. |
Sorry, Jan. On Friday last week I decided to use another approach and it worked for us. Thank you very much for your ideas, we mixed them with ours. |
FYI: #296 |
Already implemented in #296 |
This PR contains:
see #185